Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

helper-cli: Test that curations are added when creating an analyzer results from package lists #9532

Merged
merged 3 commits into from
Dec 2, 2024

Conversation

fviernau
Copy link
Member

@fviernau fviernau commented Dec 2, 2024

The test has been missing back when the functionality was added.

Done in context of: #8739.

@fviernau fviernau requested a review from a team as a code owner December 2, 2024 12:25
@fviernau fviernau force-pushed the helper-cli-package-list-test-for-curations branch from a61ea3f to e7db744 Compare December 2, 2024 12:27
The test reads the ORT configuration from the default location. If the
configuration cannot be de-serialized, the test fails.

So, isolate that test from any ORT configuration present at the default
storage location by generating an ORT configuration. The package
curation providers are set to be empty to ensure that the command does
not import any package curations (from the respective default storage
location).

Signed-off-by: Frank Viernau <[email protected]>
@fviernau fviernau force-pushed the helper-cli-package-list-test-for-curations branch from e7db744 to e03cc1c Compare December 2, 2024 12:30
@fviernau fviernau enabled auto-merge (rebase) December 2, 2024 12:30
Copy link

codecov bot commented Dec 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.99%. Comparing base (8b56475) to head (e03cc1c).
Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9532      +/-   ##
============================================
+ Coverage     67.91%   67.99%   +0.07%     
  Complexity     1290     1290              
============================================
  Files           249      249              
  Lines          8818     8818              
  Branches        917      917              
============================================
+ Hits           5989     5996       +7     
+ Misses         2440     2433       -7     
  Partials        389      389              
Flag Coverage Δ
funTest-docker 64.84% <ø> (ø)
funTest-non-docker 33.30% <ø> (+0.08%) ⬆️
test 35.82% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fviernau fviernau merged commit dd2bca5 into main Dec 2, 2024
23 checks passed
@fviernau fviernau deleted the helper-cli-package-list-test-for-curations branch December 2, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants